2. 조건식 통합하기
- 응집도가 높은 조건식을 하나의 조건식으로 구성함으로써 관리의 편의성을 높이고 추가로 함수로 추출할 수 있음
- 결과가 같은 조건식은 하나로 통합한다.
절차
- 해당 조건식들 모두에 부수효과가 없는지 확인
- 조건문 두 개를 선택하여 두 조건문의 조건식들을 논리 연산자로 결합
- 테스트
- 2~3 반복
- 합친 조건식을 함수로 추출할지 고려
예시 코드
😞 Before
if (anEmployee.seniority < 2) return 0
if (anEmployee.monthsDisabled > 12) return 0
if (anEmployee.isPartTime) return 0
😃 After
const isNotEligibleForDisability = () => {
return anEmployee.seniority < 2 || anEmployee.monthsDisabled > 12 || anEmployee.isPartTime
}
if (isNotEligibleForDisability()) return 0